-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
types: widgets: Add URL Video widget #515
types: widgets: Add URL Video widget #515
Conversation
src/components/widgets/Video.vue
Outdated
></v-text-field> | ||
<div> | ||
<span class="text-xs font-semibold leading-3 text-slate-600">Fit style</span> | ||
<Dropdown v-model="widget.options.fitStyle" :options="['cover', 'fill', 'contain']" class="max-w-[144px]" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<Dropdown v-model="widget.options.fitStyle" :options="['cover', 'fill', 'contain']" class="max-w-[144px]" /> | |
<Dropdown v-model="widget.options.fitStyle" :options="['cover', 'fill', 'contain']" variant="outlined" class="max-w-[144px]" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested, works great. The name seems confusing, maybe something like "MP4 Video Player", or "URL Video Player" or "File Video Player"
Signed-off-by: Patrick José Pereira <[email protected]>
Signed-off-by: Patrick José Pereira <[email protected]>
290d4ff
to
6b6d920
Compare
Signed-off-by: Patrick José Pereira <[email protected]>
6b6d920
to
9bb29dc
Compare
I would ask on the next PRs that we follow the current pattern of one commit per logical change. |
Can be tested with:
http://commondatastorage.googleapis.com/gtv-videos-bucket/sample/BigBuckBunny.mp4
Fix #513